-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Respond to rev2 #31
Respond to rev2 #31
Conversation
Looking at the error message in Actions I think it was because Added as a 'Suggests' with:
in latest commit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great!
Interesting error message:
https://github.com/ITSLeeds/slopes/runs/2815009282#step:9:53 Maybe worth trying with |
vignettes/roadnetworkcycling.Rmd
Outdated
# res(dem) #27m of resolution | ||
network = iow_network_segments | ||
|
||
raster::plot(dem) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice result @temospena, I didn't know this was where the hills were in IoW:
ropensci/software-review#420 (comment)
pass benchmark to a vignette and remove from README
Update argument names
@param r -> routes
in slopes.R and plot_slope.R and slope_get.R and README
@param e -> elevations (values)
in slopes.R and plot_slope.R
@param e -> dem (raster)
in slopes.R and README
lonlat
-> defaults were revised and updateddo not export plot_dz from plot_slope.R